Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement new EVM Multinode #13184

Merged

Conversation

DylanTinianov
Copy link
Contributor

@DylanTinianov DylanTinianov commented May 13, 2024

Implemented new EVM Multinode design. The Multinode is now called by chain clients to retrieve the best healthy RPC rather than performing RPC calls directly. Multinode performs verious health checks on RPCs, and in turn increases reliability. This new EVM Multinode design will also be implemented for non-EVMs chains in the future.

Multinode Design Doc: https://docs.google.com/document/d/1nvgelqk4xbznRy8zdy_KHeEjFjgKkqhSUGXSlDze7nU/edit#heading=h.4e7ng0tekbkc

@DylanTinianov DylanTinianov changed the title Multinode Interface Implement new Interfaces for Multinode May 14, 2024
Copy link
Collaborator

@dhaidashenko dhaidashenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice progress!

common/client/multi_node.go Outdated Show resolved Hide resolved
common/client/multi_node.go Show resolved Hide resolved
common/client/multi_node.go Outdated Show resolved Hide resolved
common/client/multi_node.go Outdated Show resolved Hide resolved
common/client/multi_node.go Outdated Show resolved Hide resolved
common/client/types.go Show resolved Hide resolved
common/client/types.go Outdated Show resolved Hide resolved
core/chains/evm/client/chain_client.go Outdated Show resolved Hide resolved
core/chains/evm/client/chain_client.go Outdated Show resolved Hide resolved
core/chains/evm/client/chain_client.go Show resolved Hide resolved
core/chains/evm/client/rpc_client.go Outdated Show resolved Hide resolved
core/chains/evm/client/rpc_client.go Outdated Show resolved Hide resolved
core/chains/evm/client/rpc_client.go Show resolved Hide resolved
common/client/node_lifecycle.go Outdated Show resolved Hide resolved
common/client/node_lifecycle.go Outdated Show resolved Hide resolved
common/client/multi_node.go Outdated Show resolved Hide resolved
common/client/node.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@dhaidashenko dhaidashenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to merge into BCI-3160-EVM-MultiNode-PoC

@DylanTinianov DylanTinianov merged commit 2895a65 into BCI-3160-EVM-MultiNode-PoC Jul 8, 2024
124 checks passed
@DylanTinianov DylanTinianov deleted the BCI-3160-implement-node-interface branch July 8, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants